Skip to content
This repository has been archived by the owner on May 14, 2021. It is now read-only.

Ignore non-zero exit codes on outdated #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Veraticus
Copy link

The outdated command on npm versions 4.0.0 and greater exits with exit code 1 if there are any outdated packages (see npm/npm@e768497). This change introduces a non-bang npm_shell_out command (currently used only for outdated) that will not raise on non-zero exit codes to allow for outdated package processing.

@codecov-io
Copy link

codecov-io commented Nov 28, 2016

Current coverage is 99.71% (diff: 93.33%)

Merging #4 into master will decrease coverage by 0.28%

@@           master         #4   diff @@
========================================
  Files          16         16          
  Lines         337        345     +8   
  Methods         0          0          
  Messages        0          0          
  Branches        0          0          
========================================
+ Hits          337        344     +7   
- Misses          0          1     +1   
  Partials        0          0          

Powered by Codecov. Last update 832e6a6...370876d

@luiscvalmeida
Copy link

luiscvalmeida commented Dec 15, 2016

Same thing happening to me, for now I forced the 3.10.10 npm version.

@miguelcnf
Copy link

Also having this issue so 👍

@coderanger can you have a look?

@saurabhkrjha
Copy link

saurabhkrjha commented Mar 2, 2017

facing the same issue

@rpaddock
Copy link

rpaddock commented May 5, 2017

This needs a bump. I'm currently running this patched version due to this issue. Would be great to get it reviewed and into the mainline.

@dee-kryvenko
Copy link

@coderanger any chance to get it merged or otherwise fixed any time soon?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants